Skip to content

Adding resource stat header handler#6

Open
ddattee wants to merge 2 commits into
developfrom
resource-state-header
Open

Adding resource stat header handler#6
ddattee wants to merge 2 commits into
developfrom
resource-state-header

Conversation

@ddattee

@ddattee ddattee commented Jun 15, 2018

Copy link
Copy Markdown
Owner

try out PR to check discuss if and how we can use the X-Api-Resource-State header from the API

@ronan-gloo

ronan-gloo commented Jun 15, 2018

Copy link
Copy Markdown

Adding resource stat header handler -> j'ai pas compris le but de la PR, même en regardant le code :(

  • Tu branches un middleware retry -> pourquoi la requète devrait être rejouée en fonction du header-state ?

  • Quel est le but de ce middleware, aka : quelle fonctionnalité apporte-t-il au SDK ?

@ddattee ddattee force-pushed the resource-state-header branch from ca6176f to 5f8bd85 Compare June 15, 2018 12:44
@ddattee

ddattee commented Jun 15, 2018

Copy link
Copy Markdown
Owner Author
  • retry : je ne connais pas bien les middlewares et c'est un exemple vite fait. j'ai remplace par un middleware log
  • il ajoute dans les logs l'état des ressources utilisées remonté dans l'API, pourrait servir si on ajoute un mode developpement au SDK permettant de remonter plus d'info dans les logs par exemple

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants